Skip to content

fix(ci): inherit secrets into the release workflow, serialize tag-on-main - #100

Open
thetillhoff wants to merge 1 commit into
mainfrom
fix/inherit-secrets-and-serialize
Open

fix(ci): inherit secrets into the release workflow, serialize tag-on-main#100
thetillhoff wants to merge 1 commit into
mainfrom
fix/inherit-secrets-and-serialize

Conversation

@thetillhoff

Copy link
Copy Markdown
Owner

v3.0.5 published correctly but the run is red:

env:
  GH_TOKEN:            ← empty
gh: To use GitHub CLI in a GitHub Actions workflow, set the GH_TOKEN environment variable

secrets.HOMEBREW_TAP_ACCESS_TOKEN resolved to empty: a called workflow receives no secrets unless the caller passes them. So on the automated path the homebrew tap is never updated - third instance of the same class as #99, where the workflow_call entry point behaves differently from on: push: tags:. secrets: inherit fixes it.

Second change, a race nobody has hit yet: tag-on-main had no concurrency group. Two renovate merges close together run in parallel, both read the same git describe output, and compute the same next version - one tag push wins, the other fails. Today's merges were 2-8 minutes apart, which is luck, not design. Now serialized with cancel-in-progress: false, because cancelling mid-run is how you get a tag whose release never happened.

Same fixes go to webscan.

…main

The v3.0.5 release succeeded but its run went red: the homebrew tap step got
an empty GH_TOKEN, because a called workflow receives no secrets unless the
caller passes them. Third instance of the workflow_call path behaving
differently from `on: push: tags:`.

Also adds a concurrency group. Two merges in quick succession would run in
parallel, both read the same `git describe` tag and compute the same next
version - one tag push wins, the other fails. Not cancel-in-progress: a
cancelled run leaves a tag whose release never happened.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant